Skip to content

add basic chart for the new api-syncagent#121

Merged
kcp-ci-bot merged 7 commits intomainfrom
add-api-syncagent
Jan 14, 2025
Merged

add basic chart for the new api-syncagent#121
kcp-ci-bot merged 7 commits intomainfrom
add-api-syncagent

Conversation

@xrstf
Copy link
Copy Markdown
Contributor

@xrstf xrstf commented Jan 9, 2025

This adds a simple chart to deploy an instance of https://github.com/kcp-dev/api-syncagent.

fixes #120

On-behalf-of: @SAP christoph.mewes@sap.com
@kcp-ci-bot kcp-ci-bot added dco-signoff: yes Indicates the PR's author has signed the DCO. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 9, 2025
On-behalf-of: @SAP christoph.mewes@sap.com
Comment thread charts/api-syncagent/templates/rbac.yaml Outdated
Comment thread charts/api-syncagent/templates/rbac.yaml Outdated
Comment thread charts/api-syncagent/templates/rbac.yaml Outdated
Comment thread charts/api-syncagent/templates/rbac.yaml Outdated
Comment thread charts/api-syncagent/templates/rbac.yaml Outdated
Comment thread charts/api-syncagent/templates/deployment.yaml Outdated
This offered no real benefit over someone simply having their own
rbac.yaml. Adding this as an option to the chart just means we
have to continue to support it, and why would we then only support
defining custom RBAC and not even other resources? At that point we
would become a meta-meta Helm chart and that's just not worth it.

On-behalf-of: @SAP christoph.mewes@sap.com
@xrstf xrstf force-pushed the add-api-syncagent branch from 47173e6 to f5a4c48 Compare January 10, 2025 11:25
Comment thread charts/api-syncagent/templates/rbac.yaml
xrstf added 4 commits January 13, 2025 19:16
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
@xrstf xrstf force-pushed the add-api-syncagent branch from daaa8b3 to 67af021 Compare January 13, 2025 18:33
@xrstf xrstf requested a review from embik January 13, 2025 18:34
Copy link
Copy Markdown
Member

@embik embik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Jan 14, 2025
@kcp-ci-bot
Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: c3520520a1aea58d790c46e6334b508bd32198f9

@kcp-ci-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: embik

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 14, 2025
@kcp-ci-bot kcp-ci-bot merged commit 7e464bc into main Jan 14, 2025
@kcp-ci-bot kcp-ci-bot deleted the add-api-syncagent branch January 14, 2025 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: create Helm chart for kcp-dev/api-syncagent

3 participants